Skip to content

Upgrade dependencies #821

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 7 commits into from
Nov 18, 2022
Merged

Upgrade dependencies #821

merged 7 commits into from
Nov 18, 2022

Conversation

chriskrycho
Copy link
Contributor

@chriskrycho chriskrycho commented Nov 18, 2022

As a starting point for getting the app fully up to date, this cherry-picks in the relevant changes made in #815, then layers on a further update to ember-concurrency required to get the app to actually run its tests locally. Future PRs will pick off other updates to incrementally land.

mansona and others added 5 commits November 18, 2022 11:17
(cherry picked from commit 08538ca)
(cherry picked from commit d197631)
(cherry picked from commit 9745a4b)

# Conflicts:
#	package.json
#	yarn.lock
Without these, the app wouldn't even boot properly, because it was
using the decorators in a way that *other* updates found disagreeable.
And hey: it gets us an update we want anyway!
A switch to npm (or Yarn 3 or pnpm) would all be good moves, but will
also require updating the CI config. For the sake of being able to make
consistent progress on upgrades, simply keep using Yarn for this piece.
@chriskrycho chriskrycho marked this pull request as ready for review November 18, 2022 18:47
@chriskrycho chriskrycho merged commit fdfb39e into master Nov 18, 2022
@chriskrycho chriskrycho deleted the upgrades branch November 18, 2022 19:06
@chriskrycho chriskrycho added the dependencies Pull requests that update a dependency file label Nov 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants